Skip to content

Conversation

@bevzzz
Copy link
Collaborator

@bevzzz bevzzz commented Jul 10, 2025

RQ can be configured so:

weaviate.configure.vectorIndex.quantizer.rq({ bits: 6, rescoreLimit: 20 });

When updating, it is only possible to update rescoreLimit.

@bevzzz bevzzz requested a review from tsmith023 July 10, 2025 14:40
@bevzzz bevzzz self-assigned this Jul 10, 2025
Copy link

@orca-security-eu orca-security-eu bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Orca Security Scan Summary

Status Check Issues by priority
Passed Passed Infrastructure as Code high 0   medium 0   low 0   info 0 View in Orca
Passed Passed SAST high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Secrets high 0   medium 0   low 0   info 0 View in Orca
Passed Passed Vulnerabilities high 0   medium 0   low 0   info 0 View in Orca

@bevzzz bevzzz requested a review from a team as a code owner July 10, 2025 14:50
bevzzz added 5 commits July 10, 2025 17:19
The previous commit only added a new env variable, it is not
included in the test matrix automatically.
v1.31 is now tested for node 22.x and v1.32 against the last 3 supported node versions.
Add 'muvera' config for Weaviate 1.31+.
Add 'rq' config for Weaviate 1.32+.
Update expected error string
This test fast failing with 'source not well formed' error message,
which is Weaviate's speak for 'I dont think this property is a reference'.
Still not sure why, but I figured it may have to do with how the test-suite is structured;
namely, that each test relies on the state produced/left by the previous test.

To test reference deletion, we instead simply create 2 objects,
reference one from the other and then delete that reference.
No external state.
@bevzzz bevzzz requested a review from tsmith023 July 14, 2025 13:29
@tsmith023 tsmith023 merged commit d13502f into main Jul 14, 2025
18 checks passed
@tsmith023 tsmith023 deleted the feat/rq branch July 14, 2025 15:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants